Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upload page improvements #553

Merged
merged 2 commits into from
Apr 17, 2017
Merged

Upload page improvements #553

merged 2 commits into from
Apr 17, 2017

Conversation

gbaz
Copy link
Contributor

@gbaz gbaz commented Oct 24, 2016

Some cleanup and modernization of the upload page.

Some cleanup and modernization of the upload page.
@hvr
Copy link
Member

hvr commented Mar 29, 2017

@sclv is this PR still ready to go? if so I'd like to pick it up for next weekend's small-change deployment

the old version.
</li>
cabal haddock --builddir="$dir" --for-hackage --haddock-option=--hyperlinked-source
cabal upload -d $dir/*-docs.tar.gz
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note that starting with cabal 2.0 one also needs to use the --publish flag to cabal upload (as it defaults to candidates)

accepts GHC features, it is also more picky about comment syntax than
the old version.
</li>
cabal haddock --builddir="$dir" --for-hackage --haddock-option=--hyperlinked-source
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why the builddir argument?

@gbaz
Copy link
Contributor Author

gbaz commented Mar 29, 2017

@hvr I picked up the script from one you proposed. If you'd like to make any changes to it, go for it!

@gbaz
Copy link
Contributor Author

gbaz commented Mar 29, 2017

Other than the changes you'd like to make there, I'd be happy to see this deployed.

@hvr
Copy link
Member

hvr commented Apr 17, 2017

@gbaz Unfortunately I just realised that this is one of the templates that diverge between the master and central-server branch... which makes this somewhat awkward to merge... :-/

@hvr hvr merged commit 3b928eb into master Apr 17, 2017
hvr added a commit that referenced this pull request Apr 17, 2017
@hvr hvr deleted the upload-page-changes branch April 17, 2017 13:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants